Skip to content

Conversation

@pedrodesu
Copy link
Contributor

@pedrodesu pedrodesu commented Jul 28, 2025

@pedrodesu pedrodesu requested a review from MSilva95 July 28, 2025 04:13
@pedrodesu pedrodesu self-assigned this Jul 28, 2025
@pedrodesu pedrodesu added the enhancement New feature or request label Jul 28, 2025
@pedrodesu
Copy link
Contributor Author

Note to self and to reviewer: matrix-mult and matrix-ops need fixing, even though I didn't address them during quest 8. I'll apply the fix here and in module before reviews take place.

Copy link
Member

@MSilva95 MSilva95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything looks fine, but I will mark it as requested changes since there is an issue with the "commit-stats" exercise, and it may require changes in the solution depending on your approach.

@MSilva95 MSilva95 self-requested a review September 23, 2025 10:00
@pedrodesu pedrodesu merged commit c858248 into master Sep 23, 2025
2 checks passed
@pedrodesu pedrodesu deleted the CON-3602-quest-08 branch September 23, 2025 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants